Add GetCiphertextLength for CBC, CFB, and ECB. #45003
Conversation
|
Note regarding the This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change. |
|
Tagging subscribers to this area: @bartonjs, @vcsjones, @krwq, @jeffhandley Issue DetailsAdds Contributes to #2406
|
...aries/System.Security.Cryptography.Primitives/ref/System.Security.Cryptography.Primitives.cs
Show resolved
Hide resolved
...stem.Security.Cryptography.Primitives/src/System/Security/Cryptography/SymmetricAlgorithm.cs
Outdated
Show resolved
Hide resolved
...stem.Security.Cryptography.Primitives/src/System/Security/Cryptography/SymmetricAlgorithm.cs
Outdated
Show resolved
Hide resolved
...stem.Security.Cryptography.Primitives/src/System/Security/Cryptography/SymmetricAlgorithm.cs
Show resolved
Hide resolved
...stem.Security.Cryptography.Primitives/src/System/Security/Cryptography/SymmetricAlgorithm.cs
Outdated
Show resolved
Hide resolved
* Too many empty lines.
* Change CryptographicException to InvalidOperationException.
* Add a remark that CFB accepts and valid feedback size, even if
the underlying algorithm does not support that feedback size.
|
Speaking of new APIs, are there any plans to add an API to specify padding for AesGcm and AesCcm ? |
No, because padding doesn't normally come up with those. Do you know of a case where it does? |
GCM / CCM is stream-like, so it does not require padding. One bit in, one bit out. Edit: Always 20 seconds too slow. |
ghost
locked as resolved and limited conversation to collaborators
Adds
GetCiphertextLengthpublic APIs.Contributes to #2406